Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Port Rust to DragonFlyBSD #16073

Merged
merged 3 commits into from
Jul 31, 2014
Merged

Port Rust to DragonFlyBSD #16073

merged 3 commits into from
Jul 31, 2014

Conversation

mneumann
Copy link
Contributor

Not included are two required patches:

  • LLVM: segmented stack support for DragonFly 1
  • jemalloc: simple configure patches

Not included are two required patches:

* LLVM: segmented stack support for DragonFly [1]

* jemalloc: simple configure patches

[1]: http://reviews.llvm.org/D4705
@brson
Copy link
Contributor

brson commented Jul 29, 2014

Neat! This basically looks good to me, and I'm glad that the patch itself is mostly straightforward. @alexcrichton should look too.

It's unfortunate to add more 'cfg sprawl': it's already pretty messy and is only going to get worse. Both rust-lang/rfcs#185 (factoring out a platform abstraction layer specifically for the runtime) and rust-lang/rfcs#131 (flexible target specification) could help. @mneumann since you have fresh experience with porting Rust, if you have opinions about what would help porters, you might mention them on those bugs.

@alexcrichton
Copy link
Member

I'm also glad to see that this didn't require too much platform-specific voodoo beyond plastering #[cfg] all over the place!

@mneumann
Copy link
Contributor Author

Hm, the travis build fails on spawn-stack-too-big.rs which does not even exist in my repo... other than that this should compile fine now.

bors added a commit that referenced this pull request Jul 31, 2014
Not included are two required patches:

* LLVM: segmented stack support for DragonFly [1]

* jemalloc: simple configure patches

[1]: http://reviews.llvm.org/D4705
@bors bors closed this Jul 31, 2014
@bors bors merged commit 284ffc5 into rust-lang:master Jul 31, 2014
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 11, 2023
Replace `doc_comments_and_attrs` with `collect_attrs`

fix rust-lang#16063

I looked at the other usages of `doc_comments_and_attrs` and it seems all of them are prone to ignoring inner attributes. `@Veykril` should I replace all of those with `collect_attrs` and remove `doc_comments_and_attrs` (or even `HasDocComments`) entirely?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants